Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add in-flight icing products to inline post #568

Closed
wants to merge 14 commits into from

Conversation

SamuelTrahanNOAA
Copy link
Contributor

@SamuelTrahanNOAA SamuelTrahanNOAA commented Aug 2, 2022

Description

Add in-flight icing products to the inline post.

A full description of the new feature, and status of the draft, can be found in this merged UPP PR:

NOAA-EMC/UPP#536

Issue(s) addressed

ufs-community/ufs-weather-model#1647

Testing

How were these changes tested? Results for unaffected tests do not change. New fields are created with valid content, unless library is absent, then the fields have missing values.

What compilers / HPCs was it tested with? Tests were run on hera and jet, but the target machine is WCOSS2; the library has restricted access and won't be available elsewhere.

Are the changes covered by regression tests? Yes.

Dependencies

NOAA-EMC/UPP#655

@SamuelTrahanNOAA SamuelTrahanNOAA changed the title add IFI to post add in-flight icing products to inline post Aug 2, 2022
@DusanJovic-NOAA
Copy link
Collaborator

@SamuelTrahanNOAA What's the status of this PR? I hasn't been updated for 3 months.

@SamuelTrahanNOAA SamuelTrahanNOAA marked this pull request as ready for review March 9, 2023 17:47
@SamuelTrahanNOAA
Copy link
Contributor Author

This PR is scheduled to be merged soon and nobody has reviewed it.

@SamuelTrahanNOAA
Copy link
Contributor Author

This was merged as part of #628

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants